Skip to content

Conversation

@RoxyFarhad
Copy link
Collaborator

@RoxyFarhad RoxyFarhad commented Dec 18, 2025

This PR introduces a new naming convention to support multiple environment deployments:

  • Environment Label (environment-label): The pre-defined deployment target you specify when triggering the workflow (dev, staging, prod-multitenant). This is what you select from the dropdown in GitHub Actions.
  • Environment Name (env-name): The actual environment names defined in your environments.yaml file (dev.aws, dev.azure, staging, etc.). These are the specific deployment configurations that map to the environment label.

Why we need this distinction:

  • One-to-Many Mapping: A single environment label can now map to multiple environment names (e.g., dev → dev.aws, dev.azure), enabling parallel deployments to different cloud providers
  • Non-explicit Mapping: If an environment has an env-name that is the same as the deployment target (i.e. an env-name of dev and a deployment target dev) it will automatically deploy to that environment.

This supports the desired config change of having different environments that map to a single "environment":

schema_version: "v1" # This is used to validate the file structure and is not used by the agentex CLI
environments:
  dev-aws:
    environment: "dev"
    kubernetes:
      namespace: "sgp-000-hello-acp"
    auth:
      principal:
        user_id: ...
        account_id: ...
    helm_overrides:
      ...
  dev-azure:
    environment: "dev"
    kubernetes:
      namespace: "sgp-000-hello-acp"
    auth:
      principal:
        user_id: ...
        account_id: ...
    helm_overrides:
      ...

Backwards compatability:

  • This introduces new helper methods that are used in the deployment pipelines and doesn't change any existing methods. This is purely an additive change so it will not break any older code, or anyone using these helper function sin older SDKs.

Tests against internal scale agents:

  • New Process (NB. all agents that did not deploy was not because of this new code)
  • Old Process - N.B. the deployment failed because the agent doesn't have the correct creds setup but the deployment to the cluster works correctly.

N.B. There was a large diff generated by linter for some reason

@RoxyFarhad RoxyFarhad force-pushed the RF/support-multiple-envs branch from 5927953 to f2e47c5 Compare December 18, 2025 12:10
@RoxyFarhad RoxyFarhad force-pushed the RF/support-multiple-envs branch from e2aa69e to 5259f20 Compare December 18, 2025 12:29
@RoxyFarhad RoxyFarhad force-pushed the RF/support-multiple-envs branch from 2f8c94b to 7881ec2 Compare December 18, 2025 17:39
@RoxyFarhad RoxyFarhad force-pushed the RF/support-multiple-envs branch from c55c9e4 to 0c3a8c2 Compare December 19, 2025 12:43
@RoxyFarhad RoxyFarhad requested a review from a team December 22, 2025 12:26
@RoxyFarhad RoxyFarhad force-pushed the RF/support-multiple-envs branch from aeeb44f to 9939a25 Compare January 2, 2026 15:24
@RoxyFarhad RoxyFarhad merged commit 4b91b95 into main Jan 2, 2026
29 checks passed
@RoxyFarhad RoxyFarhad deleted the RF/support-multiple-envs branch January 2, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants